few comments

lgtm

Hey Sakshi, nice work on the builder — the overall structure looks solid. One issue though:

transaction.ts wasn't updated in this PR. The setTransactionType() method (line 379) only allows AddPermissionlessValidator in its whitelist, so when the delegation builder calls setTransactionType(AddPermissionlessDelegator) during buildImplementation(), it'll throw "Transaction type is not supported". The outputs, changeOutputs, inputs, and explainTransaction getters also need cases for the delegation tx type.

The reason the existing tests don't catch this is that every build() test expects a rejection before it ever gets to setTransactionType (missing nodeID, missing UTXOs, etc.). There's no test that actually builds a complete valid delegation transaction end-to-end.

Would be good to:

1. Update transaction.ts to support AddPermissionlessDelegator in setTransactionType and the output/input/explain getters
2. Add the delegation type to the Tx union in iface.ts (it was added to SerializedTx but not Tx)
3. Add an end-to-end build test with all required params, and ideally an initBuilder round-trip test (build -> serialize -> factory.from() -> verify)

Looks good — all the issues from the previous review are addressed. transaction.ts is updated, Tx type union is fixed, and the round-trip tests with real tx data cover the deserialization path. Ship it.